Revert "tool button: Use G_DEFINE_TYPE"

This reverts commit 4e56dfad0e33187d019c44ecfb5e7e99cf2da535.

It turns out that GtkToolButton is playing games with the class
pointer inside instance_init, so leave it alone.

gtk-builder-tool: Don't break GtkHBox and GtkVBox

These classes have different defaults for ::expand than GtkBox,
which is not reflected in the declared default, so we have to
blacklist it.

gtk-demo: Fix builder example

This example is still using a GtkVBox, and it happened to be broken
by gtk-builder-tool simplify.

Updated Galician translations

Updated Galician translations

widget: Call size_allocate vfunc directly when possible

This avoids all the overhead of signal emissions when it is not used.

draw: call vfunc rather then emit signal for the common case

This avoids a lot of overhead in the common case where a signal
is not connected and we're just using the class vfunc (which is true
for all in-libgtk widgets). Additionally it makes backtraces in
debuggers and profiles much much nicer to look at.

https://bugzilla.gnome.org/show_bug.cgi?id=754986

gtk: Stop setting GDK_EXPOSURE_MASK on random widgets

These days exposure happens only on the native windows (generally the
toplevel window) and is propagated down recursively. The expose event
is only useful for backwards compat, and in fact, for double buffered
widgets we totally ignore the event (and non-double buffering breaks
on wayland).

So, by not setting the mask we avoid emitting these events and then
later ignoring them.

We still keep it on eventbox, fixed and layout as these are used
in weird ways that want backwards compat.

gdkwindow: Avoid list allocation and object refs during repaint

There is no need to ref the windows we're ignoring, so collect and ref
only the affected child windows. Also, use a on-stack array rather
than allocating a linked list.

Also, we don't need to ref during the event emissions too, as we
already hold a ref.

https://bugzilla.gnome.org/show_bug.cgi?id=754687

Updated Portuguese translation

Updated Polish translation

Update all ui files again

Replace translatable="1" by translatable="yes" so
xgettext stays happy and extracts the strings.

builder-tool: Don't simplify translatable="yes"

It turns out that xgettext does not understand translatable="1",
so don't make gtk-builder-tool produce this, even though
GtkBuilder can parse it just fine.

https://bugzilla.gnome.org/show_bug.cgi?id=754928

file chooser: make rows with popovers visible

Without this, the popovers are just dangling in the middle of
nowhere.

https://bugzilla.gnome.org/show_bug.cgi?id=754411

pixelcache: squash warnings introduced by previous commits

textview: give pixelcache access to style context

This changes textview to share the style context with the pixelcache.
Doing so allows pixel cache to optimize the surface creation and use
a CAIRO_CONTENT_COLOR instead of CAIRO_CONTENT_COLOR_ALPHA when
appropriate.

https://bugzilla.gnome.org/show_bug.cgi?id=754658

viewport: share style context with pixelcache

This allows the pixel cache to opportunistically use CAIRO_CONTENT_COLOR
instead of CAIRO_CONTENT_COLOR_ALPHA during surface creation.

https://bugzilla.gnome.org/show_bug.cgi?id=754658

pixelcache: use GtkStyleContext to determine cairo_content_t for surface

We can take a fast path if the background for a widget is opaque by using
a CAIRO_CONTENT_COLOR instead of a CAIRO_CONTENT_COLOR_ALPHA surface. Most
blit'ing backends have a fast path for this, including Pixman and Quartz.

https://bugzilla.gnome.org/show_bug.cgi?id=754658

stylecontext: add opaque background helper

This new private API, _gtk_style_context_is_background_opaque(), is meant to
be used by internal Gtk+ wigets to optimize fast paths for cases where
applicable. One such use would be to use a CAIRO_CONTENT_COLOR surface
instead of CAIRO_CONTENT_COLOR_ALPHA.

https://bugzilla.gnome.org/show_bug.cgi?id=754658

wayland: Avoid a crash with GtkGLArea

We must not call gdk_wayland_window_attach_image when
using GL for drawing, this leads to a crash.

https://bugzilla.gnome.org/show_bug.cgi?id=754770

gdkwindow: Store children list nodes in GdkWindow structure

This avoids a bunch of allocations, and additionally it has better
cache behaviour, as we don't follow pointers to the separate GList
node memory areas during traversal.

From Christian Hergert:

  This machine is a Retina mac book pro so I've been working on getting
  GtkTextView (GtkPixelCache) up to our performance level on
  X11/Wayland. I'm seeing a jump from about 43 FPS to about 50 FPS.

https://bugzilla.gnome.org/show_bug.cgi?id=754687

places sidebar: keep open variants sensitive

Add a separate action for "Open in Tab" and "Open in new window"
so we can keep them enabled when we disable "Open" for the selected
row.

https://bugzilla.gnome.org/show_bug.cgi?id=754410

Updated Slovak translation

Updated Slovak translation

Drop GtkWidgetAuxInfo

Merge it into GtkWidgetPrivate. In my measurements, about half
of all widgets have a non-default auxinfo struct, and we use this
information in size allocation, so it is nice to avoid the gdata
overhead.

Move font map to qdata

This is very rarely used, lets not reserve space for it
in every widget.

Move font options to gdata

This is very rarely used, lets not reserve space for it in
every widget.

Move action muxer to gdata

Most widgets don't have their own, and we don't access them
very frequently.

Move widget paths to gdata

We only use widget paths for a few widgets nowadays (notebook,
treeview, pathbar, combobox), so we can save some space by
not having this field in GtkWidgetPrivate.

combobox: Stop using old styling code

Instead of get_path_for_child() use CSS nodes.

Move template support to qdata

The hash table is only accessed at creation and destruction time,
and many widgets don't use templates at all, so no need to have
this permanently occupying space.

Pack GtkWidgetPrivate a bit better

This makes it shrink from 240 to 232 bytes.

Use NULL instead of g_direct_equal

GHashTable has built-in support for pointer comparison.

Move GtkWidgetAuxInfo struct to private headers

There's no API returning such a struct, it is entirely internal.
Therefore, don't expose it in public headers.

Revert "Avoid type checking overhead for resize-mode"

This reverts commit 3eacfa88f2d9c690a006de9cac1cd3c60901686c.

Apart from the patch not being correct, we don't want to expose private
structures in header files if we can avoid it.
And this type-checking overhead is not an optimization that is even
measurable.

https://bugzilla.gnome.org/show_bug.cgi?id=754932

Updated POTFILES.in

Use g_object_get_qdata instead of g_object_get_data

This is less expensive.

Intern all signal names beforehand

This avoids pointless allocations

css node: Avoid a duplicate type check

widget path: No need for atomic refcounting

We are all in one thread, anyway.

css value: No need for atomic refcounting

We are all in one thread anyway.

css section: No need for atomic refcounting

We are all in one thread, anyway.

Avoid type checking overhead for resize-mode

window: Avoid unnecessary detours

Don't call gtk_widget_get_screen on a window - it only ends up
calling gtk_window_get_screen in the end, anyway.

Remove type check from gtk_widget_get_request_mode

widget: Move _gtk_widget_peek_request_cache to private.h

css node declaration: Avoid optional out parameters

Always pass a pos to find_class, and avoid the if.

css node declaration: Fix a typo

css section: Replace g_return by gtk_internal_return

style cascade: Replace g_return by gtk_internal_return

css lookup: Replace g_return by gtk_internal return

These functions are called in an inner loop in
gtk_css_style_provider_lookup.

css selector: Use a builtin to count bits

css matcher: Avoid a redundant call

The sole caller passes an interned string, so we don't
need to call g_intern_string again.

css selector: Take advantage of interned strings

We know these strings are interned, no need to use strcmp or
g_str_hash for hem.

bitmask: Trivial formatting fix

bitmask: Optimize no-op resizes

Statistics for the gtk3-demo listbox example show that the
vast majority of calls to _gtk_allocated_bitmask_resize go
from a size of 2 to 2. Don't needlessly call realloc() in
this case.

bitmask: Optimize intersect

The functions was written in a way that would possibly
resize the mask twice, which is not necessary.

widget-factory: Simplify the ui file

gtk-demo: Simplify ui files

Run gtk-builder-tool simplify over all templates

gtk-builder-tool: Canonicalize booleans

Always write booleans as '0' or '1'.

builder: Minor tweak

widgetpath: Implement using GtkCssNodeDeclaration

This removes a bunch of code but more importantly allows providing the
declaration in CSS matchers.

cssnode: Move clear_classes() into the NodeDeclaration

The previous code was crashing when used as the returned classes array
would have been invalid after the first deletion. So if a 2nd class
would be deleted, invalid memory might have been referenced.

stylecontext: Fix list_classes() after optimization

Otherwise a junk value is returned.

widgetpath: Remove bad function prototypes

iconcache: Guard deprecated gdk-pixbuf APIs

cssnode: Optimize listing of classes

Avoid creating the GList of strings when the GQuarks
will just be determined again. Instead have
gtk_style_context_list_classes() do it specifically.

container: Don't add classes twice

_gtk_widget_create_path() already adds the style classes.

gtkplug.c: remove draw function, not needed

Background will be rendered by parent class - GtkWindow.

https://bugzilla.gnome.org/show_bug.cgi?id=754854

Improve printer distinction for option retrieval

In order to retrieve the user options for a printer, the respective
printer name is used.
This fixes the comparison of printer names to avoid that the options of
another printer are accidently read whose name starts with the same
letters, but is longer (e.g. "myprinterlongername" instead of
"myprinter").

This fixes Bug 753628.

HighContrast: make sidebar list rows taller

sync with Adwaita

https://bugzilla.gnome.org/show_bug.cgi?id=754866

GtkFileSystem: Add to the list of remote filesystems

Add afp and google-drive to the list of remote filesystems.

https://bugzilla.gnome.org/show_bug.cgi?id=754746

Updated Slovak translation

GtkWidget: Fix typo in #ifdef

It's ENABLE, not ENABEL. Also fix the corresponding #endif comments.

GtkContainer: Fix paramter name in doc comment

Optimize gtk_css_node_declaration_has_class

This function is getting called a lot. Statistics for the gtk3-demo
listbox example show most calls with 0-4 classes. Unrolling the cases
a bit brings the instruction count in callgrind from 93M to 52M.

More bit manipulation speedups

The same trick that was applied to _gtk_css_change_for_child in
the previous commit can be applied to _gtk_css_change_for_sibling
as well, and that is what this commit does.

With both functions converted, gtk_css_change_translate is no
longer needed and gets dropped.

Manipulate bits more efficiently

The function to translate GtkCssChange enum values to the PARENT
ones is called very frequently. This patch speeds it up tremendously.
The callgrind instruction count for this function in the listbox
demo goes from 108M to 7M.

match_all: Only allocate an array when needed

My statistics show that more than half of all calls end up
with 0 matches, so we can avoid some overhead by not allocating
an array at all in this case.

Handle match_all returning NULL

This is in preparation for not allocating an array at all if there
are no matches.

Optimize gtk_css_selector_match_all

We are dealing with really short lists here.
95% are < 10 matches, and the longest I've been able to record was 19.
So just do away with the hash table and do sorted insertion in
the array directly.

x11: Declare variable for the inner loop

Otherwise the outer loop control variable is messed up, and we end
up with uninitialized axes if there were any more valuators after
the XIKeyClass one.

This bug was sneakily introduced by fdb9a8e14, many thanks to
Carlos Soriano for helping spot the source of this bug.

https://bugzilla.gnome.org/show_bug.cgi?id=753431

Adwaita: special case selection mode lists checks

...otherwise the special selection mode 40x40px checks are used
shrinked to 16x16.
See https://bugzilla.gnome.org/show_bug.cgi?id=754610

Update the gtk3-demo man page

We have options now !

gtk-demo:Add a --list option

This just shows the available examples and then exists.

gtk-demo: Make --autoquit a simple option

No real need to specify the amount of delay, just make this
a plain boolean.

Use G_ENABLE_CONSISTENCY_CHECKS for template finalization checks

css selector: Use the quarks we have

Unless I'm missing something, there is no reason to go from
quarks back to strings to implement hasing or comparison.

style provider private: Use gtk_internal_return_if_fail

Clean up debug features

Introduce a GTK_DEBUG_CHECK() macro and use it to check for
GTK_DEBUG flags everywhere. Also guard all such places by

widget: Use G_ENABLE_CONSISTENCY_CHECKS for invariants checking

The logic here is that G_ENABLE_DEBUG is for compiling out
debug spew that can be triggered at runtime with the GTK_DEBUG
environment variable, while G_ENABLE_CONSISTENCY_CHECKS is for
consistency checks that are applied unconditionally.

size request: Redo the recursion checks

Use G_ENABLE_CONSISTENCY checks for the recursion checks
in gtksizerequest.c

Introduce G_ENABLE_CONSISTENCY_CHECKS

Use a separate G_ENABLE_CONSISTENCY_CHECKS define to guard internal
consistency checks that are applied unconditionally if they are enabled,
such as the widget invariants checking. Interactive debug spew that can
be triggered at runtime with the GTK_DEBUG environment variable is still
guarded by the G_ENABLE_DEBUG define.

The mapping from enable-debug levels to defines is as follows:

yes:     G_ENABLE_DEBUG G_ENABLE_CONSISTENCY_CHECKS
minimum: G_ENABLE_DEBUG G_DISABLE_CAST_CHECKS
no:      G_DISABLE_CAST_CHECKS G_DISABLE_ASSERT G_DISABLE_CHECKS

notebook: Drop use of G_DISABLE_CHECKS

Rewrite the one check that the G_DISABLE_CHECKS macro was used for
in terms of g_return_if_fail.

Drop an unused warning

It has been ifzeroed for many, many years.

Use a getter for widget direction

settings: Reuse a value

We've already gotten the value type, use it.

builder: Avoid some type-checking overhead

settings: Avoid a redundant check

g_value_transform already returns whether it was successful
or not, so no need to call g_value_type_transformable beforehand.